Skip to content

Fix CONDA_PREFIX for RPC debug path#33

Merged
aahrun merged 1 commit intomodular:mainfrom
aahrun:fix-conda-prefix-rpc
Dec 5, 2025
Merged

Fix CONDA_PREFIX for RPC debug path#33
aahrun merged 1 commit intomodular:mainfrom
aahrun:fix-conda-prefix-rpc

Conversation

@aahrun
Copy link
Contributor

@aahrun aahrun commented Dec 4, 2025

prefixPath was not being passed properly when debugging via RPC, which causes CONDA_PREFIX to be empty and an incorrect path used when attempting to launch mojo-lldb-dap. This breaks debugging via mojo debug --vscode in conda environments.

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@aahrun
Copy link
Contributor Author

aahrun commented Dec 4, 2025

I have read the CLA Document and I hereby sign the CLA

modular-cla-bot bot added a commit to modular/cla that referenced this pull request Dec 4, 2025
Copy link

@alycm alycm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

prefixPath was not being passed properly when debugging via RPC,
which causes CONDA_PREFIX to be empty and an incorrect path used
when attempting to launch mojo-lldb-dap. This breaks debugging
via mojo debug --vscode in conda environments.
@aahrun aahrun force-pushed the fix-conda-prefix-rpc branch from e49d813 to a478738 Compare December 5, 2025 11:04
@aahrun aahrun merged commit a3e8ff3 into modular:main Dec 5, 2025
3 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2025
@aahrun aahrun deleted the fix-conda-prefix-rpc branch December 5, 2025 11:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants